-
Notifications
You must be signed in to change notification settings - Fork 486
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Docs: CRD reference topic edits #2453
Conversation
cmd/agent-operator/DEVELOPERS.md
Outdated
Use the following to run the script in a container: | ||
|
||
``` | ||
docker build -t grafana/agent-build-image:0.18.0 ./build-image |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This version number is too specific, maybe we could just use latests
as a tag.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! Left one minor comment, but the rest LGTM
cmd/agent-operator/DEVELOPERS.md
Outdated
Use the following to run the script in a container: | ||
|
||
``` | ||
docker build -t grafana/agent-build-image:0.18.0 ./build-image |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
docker build -t grafana/agent-build-image:0.18.0 ./build-image | |
docker build -t grafana/agent-build-image:latest ./build-image |
cmd/agent-operator/DEVELOPERS.md
Outdated
|
||
``` | ||
docker build -t grafana/agent-build-image:0.18.0 ./build-image | ||
BUILD_IMAGE=grafana/agent-build-image:0.18.0 USE_CONTAINER=1 make generate-crds |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
BUILD_IMAGE=grafana/agent-build-image:0.18.0 USE_CONTAINER=1 make generate-crds | |
BUILD_IMAGE=grafana/agent-build-image:latest USE_CONTAINER=1 make generate-crds |
PR Description
Edited
agent/operator/apis/monitoring/v1alpha1/
deployment.go
doc.go
group.go
types_integrations.go
types_logs.go
Will finish editing topics in this issue: #2452
Which issue(s) this PR fixes
Fixes #2451
Notes to the Reviewer
PR Checklist